Skip to content

move /community-images to /static/images/community and added gatsby-source-filesystem for /static/images#7494

Open
saurabhraghuvanshii wants to merge 15 commits intolayer5io:masterfrom
saurabhraghuvanshii:images
Open

move /community-images to /static/images/community and added gatsby-source-filesystem for /static/images#7494
saurabhraghuvanshii wants to merge 15 commits intolayer5io:masterfrom
saurabhraghuvanshii:images

Conversation

@saurabhraghuvanshii
Copy link
Copy Markdown
Member

Description

This PR fixes #

Notes for Reviewers

Signed commits

  • Yes, I signed my commits.

Signed-off-by: saurabhraghuvanshii <saurabhsraghuvanshi@gmail.com>
@l5io
Copy link
Copy Markdown
Member

l5io commented Mar 11, 2026

🚀 Preview for commit 1688d52 at: https://69b1eba8d782be3f10f5b97b--layer5.netlify.app

Copy link
Copy Markdown
Member

@leecalcote leecalcote left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

resolve: "gatsby-source-filesystem"... don't we have this already?

@l5io
Copy link
Copy Markdown
Member

l5io commented Mar 11, 2026

🚀 Preview for commit 5b37bbe at: https://69b1f7e0426f7f7cf5a7ebfb--layer5.netlify.app

@saurabhraghuvanshii saurabhraghuvanshii changed the title move /community-images to /static/images/community move /community-images to /static/images/community and added gatsby-source-filesystem for /static/images Mar 11, 2026
Signed-off-by: saurabhraghuvanshii <saurabhsraghuvanshi@gmail.com>
@l5io
Copy link
Copy Markdown
Member

l5io commented Mar 12, 2026

🚀 Preview for commit efa1e8a at: https://69b284a2625dbe6c2b858863--layer5.netlify.app

Signed-off-by: saurabhraghuvanshii <saurabhsraghuvanshi@gmail.com>
@l5io
Copy link
Copy Markdown
Member

l5io commented Mar 12, 2026

🚀 Preview for commit e5fa67d at: https://69b29132883ee2c15702d351--layer5.netlify.app

@l5io
Copy link
Copy Markdown
Member

l5io commented Mar 12, 2026

🚀 Preview for commit 12ff0e6 at: https://69b315d6c5f3ba4cb7d16092--layer5.netlify.app

@l5io
Copy link
Copy Markdown
Member

l5io commented Mar 13, 2026

🚀 Preview for commit 1691458 at: https://69b4552689c85a2dbc3e7b61--layer5.netlify.app

Signed-off-by: saurabhraghuvanshii <saurabhsraghuvanshi@gmail.com>
@l5io
Copy link
Copy Markdown
Member

l5io commented Mar 13, 2026

🚀 Preview for commit 1aaaecf at: https://69b462063a6979292f489e71--layer5.netlify.app

@l5io
Copy link
Copy Markdown
Member

l5io commented Mar 14, 2026

🚀 Preview for commit 3ca92c9 at: https://69b5735c7a8bf7da89812678--layer5.netlify.app

@l5io
Copy link
Copy Markdown
Member

l5io commented Mar 20, 2026

🚀 Preview for commit 0584eab at: https://69bdb26459f45b945918912f--layer5.netlify.app

@l5io
Copy link
Copy Markdown
Member

l5io commented Mar 23, 2026

🚀 Preview for commit 921237c at: https://69c17293df32c0114b9daa92--layer5.netlify.app

@l5io
Copy link
Copy Markdown
Member

l5io commented Mar 26, 2026

🚀 Preview for commit 8e7251c at: https://69c4fa577027c461ba82af5b--layer5.netlify.app

@arjunmehta-git
Copy link
Copy Markdown

Good progress on this asset migration, @saurabhraghuvanshii. Consolidating community images under static/images/community/ is the right long-term pattern — it makes images accessible via static URL and simplifies the GraphQL source configuration.

There are still open review threads from the earlier rounds that need to be resolved before merge. Specifically:

  • Duplicate gatsby-source-filesystem — My earlier question about whether we already have this filesystem source configured remains unanswered. Please check gatsby-config.js carefully — adding a duplicate source can cause unexpected build behavior or doubled GraphQL nodes.
  • Resolve all open threads — Several inline review comments from earlier commits appear unresolved. Please mark each thread as resolved once you've addressed it, so reviewers can see the current state at a glance.
  • Merge conflicts — Check whether the branch is still current with master. Several subsequent PRs have landed since this was opened and the branch may need a rebase.

The mechanical work of the rename (35 files moved to the correct location) is solid. Get the outstanding review items closed out and this can move forward.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants